Skip to content

Conversation

@LSPECTRONIZTAR
Copy link

I made it so that the parse JSON into block (now renamed to set to JSON) can support parsed JSON objects and the stringify block (now renamed to get as JSON) can also return parsed objects if the project creator wishes (for speed-increasing purposes). However, the return option is set to stringified by default to stay true to the old version.

I also added to the Scratch.translate.setup for the new blocks and made sure that the extension update is completely compatible with any projects that might use the previous version.

Please I need this so my 3D physics engine won't lag 🙏

@github-actions github-actions bot added the pr: change existing extension Pull requests that change an existing extension label Nov 28, 2025
@PPPDUD
Copy link
Member

PPPDUD commented Nov 28, 2025

!format

@PPPDUD
Copy link
Member

PPPDUD commented Nov 28, 2025

Please do not use Scratch.translate.setup. It's deprecated not recommended and makes your code harder to read.

@CST1229
Copy link
Collaborator

CST1229 commented Nov 28, 2025

Please do not use Scratch.translate.setup. It's deprecated and makes your code harder to read.

not deprecated, just not needed for this repo, as the build process adds it automatically.

i think what happened is that you pasted the code directly from extensions.turbowarp.org/vercte/dictionaries.js instead of the source code, which includes said generated Scratch.translate.setup calls.

// ID: verctedictionaries
// Description: Use the power of dictionaries in your project.
// By: Vercte <https://scratch.mit.edu/users/lolecksdeehaha/>
// By: Vercte <https://scratch.mit.edu/users/lolecksdeehaha/>, LSPECTRONIZTAR <https://scratch.mit.edu/users/LSPECTRONIZTAR/>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comma won't work. Need second line

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay I'll fix that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: change existing extension Pull requests that change an existing extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants